Skip to content

Rust::com General Optimization and cleanup of RUST-API#276

Open
bharatGoswami8 wants to merge 6 commits intoeclipse-score:mainfrom
bharatGoswami8:com_api_optimization
Open

Rust::com General Optimization and cleanup of RUST-API#276
bharatGoswami8 wants to merge 6 commits intoeclipse-score:mainfrom
bharatGoswami8:com_api_optimization

Conversation

@bharatGoswami8
Copy link
Copy Markdown
Contributor

@bharatGoswami8 bharatGoswami8 commented Apr 8, 2026

  • cleanup for internal type naming of rust apis
  • Updated init async flag
  • added NonNull type for internal types of consumer and producer

@bharatGoswami8 bharatGoswami8 self-assigned this Apr 8, 2026
@bharatGoswami8 bharatGoswami8 changed the title Rust::com Rust Api optimization Rust::com General Optimization and cleanup of RUST-API Apr 8, 2026
@bharatGoswami8 bharatGoswami8 force-pushed the com_api_optimization branch 3 times, most recently from a8502a3 to 9219844 Compare April 10, 2026 11:14
@bharatGoswami8 bharatGoswami8 marked this pull request as ready for review April 10, 2026 11:15
@bharatGoswami8 bharatGoswami8 force-pushed the com_api_optimization branch 3 times, most recently from 7bd3067 to 6fb410c Compare April 13, 2026 04:21
Copy link
Copy Markdown

@rpreddyhv rpreddyhv left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

looks okay to me.

Comment thread score/mw/com/impl/rust/com-api/com-api-runtime-lola/consumer.rs
Comment thread score/mw/com/impl/rust/com-api/com-api-runtime-lola/consumer.rs Outdated
Comment thread score/mw/com/impl/rust/com-api/com-api-runtime-lola/producer.rs
Comment thread score/mw/com/impl/rust/com-api/com-api-runtime-lola/producer.rs
Comment thread score/mw/com/impl/rust/com-api/com-api-runtime-lola/producer.rs Outdated
* Changed init flag from automic to std::once
* Changed NativeSkeletonHandle to NonNull
* Updated Naming for Lola and Mock runtime internal type
* Consumer proxy and event type field updated to NonNull
* Updated receive algo document
Copy link
Copy Markdown

@arvindr19 arvindr19 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LTGM :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants